[16_0_X] Add time info to Run3ScoutingHBHERecHit#49718
Conversation
…roducer accordingly
|
A new Pull Request was created by @patinkaew for CMSSW_16_0_X. It involves the following packages:
@Dr15Jones, @Martin-Grunewald, @cmsbuild, @makortel, @mmusich, @smuzaffar can you please review it and eventually sign? Thanks. cms-bot commands are listed here
|
|
cms-bot internal usage |
|
Does this require a backport of #49702 ? |
|
backport of #49715 |
|
type hlt-integration |
ultimately yes. |
|
@cmsbuild, please test |
|
+1 Size: This PR adds an extra 16KB to repository Comparison SummarySummary:
|
|
I guess #49715 (comment) applies also here. |
Yes. |
|
Pull request #49718 was updated. @Dr15Jones, @Martin-Grunewald, @cmsbuild, @makortel, @mmusich, @smuzaffar can you please check and sign again. |
|
@cmsbuild, please test |
|
test parameters:
|
|
@cmsbuild, please test |
|
-1 Failed Tests: UnitTests Failed Unit TestsI found 1 errors in the following unit tests: ---> test TestDQMOnlineClient-scouting_dqm_sourceclient had ERRORS Comparison SummarySummary:
|
|
test parameters:
|
|
@cmsbuild, please test |
|
+1 Size: This PR adds an extra 16KB to repository Comparison SummarySummary:
|
|
REMINDER @sextonkennedy, @mandrenguyen, @ftenchini: This PR was tested with #49767, please check if they should be merged together |
|
@cms-sw/core-l2 since #49767 and cms-sw/cmsdist#10272 already got merged (and they depend on this) can we have this PR merged ? |
|
+core
Fine for me, although the |
|
This pull request is fully signed and it will be integrated in one of the next CMSSW_16_0_X IBs (tests are also fine) and once validation in the development release cycle CMSSW_16_1_X is complete. This pull request will now be reviewed by the release team before it's merged. @sextonkennedy, @ftenchini, @mandrenguyen (and backports should be raised in the release meeting by the corresponding L2) |
|
+1 |
Backport of #49715 to 16_0_X for 2026 data-taking.
FYI: @silviodonato @wang-hui